Skip to content

Allow dead_code on AlignedType#689

Merged
apoelstra merged 1 commit intorust-bitcoin:masterfrom
tcharding:03-27-allow-dead-code
Mar 26, 2024
Merged

Allow dead_code on AlignedType#689
apoelstra merged 1 commit intorust-bitcoin:masterfrom
tcharding:03-27-allow-dead-code

Conversation

@tcharding
Copy link
Copy Markdown
Member

We use the AlignedType and take a pointer to its inner data, never access the data directly - this confuses clippy causing a "field is never used" warning.

Shoosh the lint and add a code comment explaining why.

We use the `AlignedType` and take a pointer to its inner data, never
access the data directly - this confuses clippy causing a "field is
never used" warning.

Shoosh the lint and add a code comment explaining why.
Copy link
Copy Markdown
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 873ad47

@apoelstra apoelstra merged commit 5fa3623 into rust-bitcoin:master Mar 26, 2024
@tcharding tcharding deleted the 03-27-allow-dead-code branch March 26, 2024 23:16
chain-forgexcr45 added a commit to chain-forgexcr45/rust-secp256k1 that referenced this pull request Sep 28, 2025
873ad47343f3dba2fd9f7f9b9944a6214e656512 Allow dead_code on AlignedType (Tobin C. Harding)

Pull request description:

  We use the `AlignedType` and take a pointer to its inner data, never access the data directly - this confuses clippy causing a "field is never used" warning.

  Shoosh the lint and add a code comment explaining why.

ACKs for top commit:
  apoelstra:
    ACK 873ad47343f3dba2fd9f7f9b9944a6214e656512

Tree-SHA512: 2a383aed169edf247def5c291bc2d8177cc09412b82bb47bb514e0929ae89d689102e646dcbbffc6fd4caa1ce1430420a63c7ceb3428ad9b05230a93b7f14d16
william2332-limf added a commit to william2332-limf/rust-secp256k1 that referenced this pull request Oct 2, 2025
873ad47343f3dba2fd9f7f9b9944a6214e656512 Allow dead_code on AlignedType (Tobin C. Harding)

Pull request description:

  We use the `AlignedType` and take a pointer to its inner data, never access the data directly - this confuses clippy causing a "field is never used" warning.

  Shoosh the lint and add a code comment explaining why.

ACKs for top commit:
  apoelstra:
    ACK 873ad47343f3dba2fd9f7f9b9944a6214e656512

Tree-SHA512: 2a383aed169edf247def5c291bc2d8177cc09412b82bb47bb514e0929ae89d689102e646dcbbffc6fd4caa1ce1430420a63c7ceb3428ad9b05230a93b7f14d16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants